Skip to content

fix(ci): ignore superseded GitHub Actions runs - #1152

Open
Archkon wants to merge 1 commit into
nodejs:mainfrom
Archkon:patch
Open

fix(ci): ignore superseded GitHub Actions runs#1152
Archkon wants to merge 1 commit into
nodejs:mainfrom
Archkon:patch

Conversation

@Archkon

@Archkon Archkon commented Aug 8, 2026

Copy link
Copy Markdown

A commit can have multiple check suites for the same workflow and event when a branch is force-pushed away from and then back to the same SHA.

Select the suite with the highest workflow run number and request only the latest check runs to avoid reporting superseded CI failures.

Fixes: #1151

@Archkon

Archkon commented Aug 8, 2026

Copy link
Copy Markdown
Author

ping @Renegade334 @aduh95 @avivkeller
Please take a look at this. This is really a must to land the associated pr from commit-queue failed

@avivkeller

Copy link
Copy Markdown
Member

Can you add a test?

@Archkon
Archkon force-pushed the patch branch 2 times, most recently from 99a586c to e1d5ae6 Compare August 10, 2026 04:19
A commit can have multiple check suites for the same workflow and event
when a branch is force-pushed away from and then back to the same SHA.

Select the suite with the highest workflow run number and request only
the latest check runs to avoid reporting superseded CI failures.

Signed-off-by: Archkon <180910180+Archkon@users.noreply.github.com>
@Archkon

Archkon commented Aug 10, 2026

Copy link
Copy Markdown
Author

Done,@avivkeller Please review this again

@codecov

codecov Bot commented Aug 10, 2026

Copy link
Copy Markdown

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 57.25%. Comparing base (22b9f8f) to head (d5e53bb).

Additional details and impacted files
@@            Coverage Diff             @@
##             main    #1152      +/-   ##
==========================================
+ Coverage   57.17%   57.25%   +0.07%     
==========================================
  Files          53       53              
  Lines       10394    10413      +19     
==========================================
+ Hits         5943     5962      +19     
  Misses       4451     4451              

☔ View full report in Codecov by Harness.
📢 Have feedback on the report? Share it here.

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.
  • 📦 JS Bundle Analysis: Save yourself from yourself by tracking and limiting bundle sizes in JS merges.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

pr_checker: ignore superseded GitHub Actions check suites when a head SHA is reused

2 participants